Skip to content

Libfido2 workflow#179

Merged
padelsbach merged 1 commit into
wolfSSL:masterfrom
JeremiahM37:libfido2
Jun 18, 2025
Merged

Libfido2 workflow#179
padelsbach merged 1 commit into
wolfSSL:masterfrom
JeremiahM37:libfido2

Conversation

@JeremiahM37
Copy link
Copy Markdown
Contributor

Added yaml workflow for libfido2. All checks pass and it seems to work with wolfProvider as expected.

I did exclude one test, regress_dev, because it tests libfido2's ability to communicate with physical FIDO devices, which fails due to environmental factors. I don't think it will impact libfido2's core cryptographic functionality or its integration with wolfProvider as all other test pass and WOLFPROV_FORCE_FAIL=1 still causes most of the other tests to fail as it should.

Comment thread .github/workflows/libfido2.yml Outdated
Comment thread .github/workflows/libfido2.yml Outdated
Comment thread .github/workflows/libfido2.yml Outdated
Comment thread .github/scripts/check-workflow-result.sh Outdated
@JeremiahM37 JeremiahM37 force-pushed the libfido2 branch 2 times, most recently from 21c571e to 8b944a1 Compare June 17, 2025 20:33
Comment thread .github/scripts/check-workflow-result.sh Outdated
Comment thread .github/workflows/libfido2.yml Outdated
@JeremiahM37 JeremiahM37 force-pushed the libfido2 branch 5 times, most recently from 4b2d78c to e0d053f Compare June 18, 2025 19:56
Comment thread .github/workflows/libfido2.yml Outdated
Comment thread .github/scripts/check-workflow-result.sh Outdated
@JeremiahM37 JeremiahM37 force-pushed the libfido2 branch 2 times, most recently from dc5b06f to 4521014 Compare June 18, 2025 20:56
padelsbach
padelsbach previously approved these changes Jun 18, 2025
Copy link
Copy Markdown
Contributor

@padelsbach padelsbach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice job, thanks

@padelsbach padelsbach merged commit 2e3e009 into wolfSSL:master Jun 18, 2025
143 of 146 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants